Repository health decreased by 0.04% when pulling a916f87 on issues/827-message-manager-improvements into 1b8dd29 on master.

• 1 new problem was found (including 0 errors and 1 code smell).
• No problems were fixed.

Coverage remained the same at 98.66% when pulling a916f87 on issues/827-message-manager-improvements into 1b8dd29 on master.

@wrightmartin I tried to cherry pick the checkbox part of this out as a separate change, but I'm not sure how it's meant to work. In the current version, the eye is a button, so clicking it will actually submit the form, but does changing it to a checkbox mean we're going to need to actually add a submit button beside each one as well? (Each td is basically its own separate form at the minute, so we can't even just have a single submit button without changing how the whole thing works, AFAICS)

@tmtmtmtm hmm, no, I think it would work best as it was before, the checkbox should submit. @chrismytton @duncanparkes any ideas if this is possible?

@tmtmtmtm @wrightmartin A <button> without a type attribute (as the page had before) will submit the form it's part of. If you want the same from a checkbox, you'll need to do it with javascript, and provide an actual submit button too as a fallback for non-javascript situations, like:

<form>
  <input type="checkbox">
  <button type="submit">Save</button>
</form>
<script>
  $(function(){
    $(':checkbox').on('change', function(){
      $(this).parents('form').submit();
    }).siblings('button').hide();
  });
</script>

Or something.

I talked to Chris about this on the train this morning, and the best we could come up with that would actually still work plausibly well without JS was to have the default version be the button (as the current live version), and have the JS replace that with the checkbox, that it then also traps and auto-submits.